Skip to content

add unify tag#1697

Merged
tgberkeley merged 3 commits intomainfrom
unify-tag
Dec 1, 2025
Merged

add unify tag#1697
tgberkeley merged 3 commits intomainfrom
unify-tag

Conversation

@tgberkeley
Copy link
Copy Markdown
Collaborator

No description provided.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Dec 1, 2025

Greptile Overview

Greptile Summary

Added Unify tracking integration to the website by importing and calling get_unify_trackers() in the pixel trackers list.

  • Imported get_unify_trackers from reflex_ui.blocks.telemetry
  • Added tracker to get_pixel_website_trackers() return list

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • Simple, straightforward addition of a new tracking integration following existing patterns. No logic changes, no security concerns, and consistent with how other trackers are implemented
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
pcweb/telemetry/pixels.py 5/5 Added Unify tracker import and function call to website trackers list

Sequence Diagram

sequenceDiagram
    participant App as Reflex App (pcweb.py)
    participant Pixels as pixels.py
    participant RefUI as reflex_ui.blocks.telemetry
    participant Browser as User Browser
    
    App->>Pixels: get_pixel_website_trackers()
    Pixels->>RefUI: get_common_room_trackers()
    RefUI-->>Pixels: component
    Pixels->>RefUI: get_google_analytics_trackers()
    RefUI-->>Pixels: [components]
    Pixels->>RefUI: gtag_report_conversion()
    RefUI-->>Pixels: component
    Pixels->>RefUI: get_clearbit_trackers()
    RefUI-->>Pixels: component
    Pixels->>RefUI: get_posthog_trackers()
    RefUI-->>Pixels: component
    Pixels->>RefUI: get_rb2b_trackers()
    RefUI-->>Pixels: [components]
    Pixels->>RefUI: get_unify_trackers()
    Note over Pixels,RefUI: NEW: Unify tracker added
    RefUI-->>Pixels: component
    Pixels-->>App: [all tracker components]
    App->>Browser: Render with tracking tags
Loading

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@tgberkeley tgberkeley merged commit 84a9f66 into main Dec 1, 2025
11 of 12 checks passed
@tgberkeley tgberkeley deleted the unify-tag branch December 1, 2025 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants